Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor v1alpha2 Linux namespace definitions #826

Merged

Conversation

chrisdoherty4
Copy link
Member

There are a handful of Linux namespaces (see https://man7.org/linux/man-pages/man7/namespaces.7.html). Containers can be configured to run inside the bounds of precreated namespaces. The original v1alpha2 API allowed users to specify a network namespace only, but we have already acquired a use-case for the PID namespace.

This change refactors the v1alpha2 API (unreleased) to accommodate future namespace customization.

@chrisdoherty4 chrisdoherty4 force-pushed the feature/refactor-template-namespaces branch from a3c67b9 to fb62c3d Compare October 24, 2023 21:33
@chrisdoherty4 chrisdoherty4 force-pushed the feature/refactor-template-namespaces branch from fb62c3d to cfd9232 Compare October 24, 2023 21:36
@codecov
Copy link

codecov bot commented Oct 24, 2023

Codecov Report

Merging #826 (3cfb681) into main (e1e969b) will not change coverage.
The diff coverage is n/a.

❗ Current head 3cfb681 differs from pull request most recent head 23df80e. Consider uploading reports for the commit 23df80e to get more accurate results

@@           Coverage Diff           @@
##             main     #826   +/-   ##
=======================================
  Coverage   51.10%   51.10%           
=======================================
  Files          33       33           
  Lines        1454     1454           
=======================================
  Hits          743      743           
  Misses        665      665           
  Partials       46       46           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

api/v1alpha2/template.go Outdated Show resolved Hide resolved
api/v1alpha2/template.go Show resolved Hide resolved
@chrisdoherty4 chrisdoherty4 force-pushed the feature/refactor-template-namespaces branch 2 times, most recently from a42b9aa to 92af321 Compare October 25, 2023 11:53
@jacobweinstock jacobweinstock force-pushed the feature/refactor-template-namespaces branch from 92af321 to 23df80e Compare October 25, 2023 15:08
Copy link
Member

@moadqassem moadqassem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@chrisdoherty4 chrisdoherty4 added the ready-to-merge Signal to Mergify to merge the PR. label Oct 25, 2023
@mergify mergify bot merged commit 029b5d6 into tinkerbell:main Oct 25, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Signal to Mergify to merge the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants